Skip to content

Correct k8s rebalance delay advice [CTT-820]#2090

Merged
JamesHazelcast merged 2 commits intohazelcast:mainfrom
JamesHazelcast:fix/ctt-820
Feb 9, 2026
Merged

Correct k8s rebalance delay advice [CTT-820]#2090
JamesHazelcast merged 2 commits intohazelcast:mainfrom
JamesHazelcast:fix/ctt-820

Conversation

@JamesHazelcast
Copy link
Contributor

@JamesHazelcast JamesHazelcast commented Feb 2, 2026

configuring-persistence conflicted with kubernetes-persistence when discussing rebalance-delay-seconds usage. The correct advice is from kubernetes-persistence.

Fixes https://hazelcast.atlassian.net/browse/CTT-820

@JamesHazelcast
Correct k8s rebalance delay advice [CTT-820]
53a223f 
`configuring-persistence` conflicted with `kubernetes-persistence` when discussing `rebalance-delay-seconds` usage. The correct advice is from `kubernetes-persistence`.

Fixes https://hazelcast.atlassian.net/browse/CTT-820
@JamesHazelcast JamesHazelcast self-assigned this Feb 2, 2026
@JamesHazelcast JamesHazelcast requested a review from a team as a code owner February 2, 2026 20:23
@netlify
Copy link

netlify bot commented Feb 2, 2026 •
edited
Loading

✅ Deploy Preview for hardcore-allen-f5257d ready!

Name Link
🔨 Latest commit 27d8b62
🔍 Latest deploy log https://app.netlify.com/projects/hardcore-allen-f5257d/deploys/6981df790fe35000086e23fa
😎 Deploy Preview https://deploy-preview-2090--hardcore-allen-f5257d.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Rob-Hazelcast
Rob-Hazelcast approved these changes Feb 3, 2026
View reviewed changes
Copy link
Contributor

@Rob-Hazelcast Rob-Hazelcast left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Possibly "warning" is too strong (I don't know what happens if you do configure a delay on K8s), but I thought it sensible to put the "don't configure this under these circumstances" statements together.

And while it's good to avoid duplication, I wanted to include the main point here rather than just linking to the other section for more info (without noting that the advice conflicts, rather than being purely supplementary/optional).

@JamesHazelcast
Copy link
Contributor Author

JamesHazelcast commented Feb 3, 2026

Possibly "warning" is too strong (I don't know what happens if you do configure a delay on K8s), but I thought it sensible to put the "don't configure this under these circumstances" statements together.

And while it's good to avoid duplication, I wanted to include the main point here rather than just linking to the other section for more info (without noting that the advice conflicts, rather than being purely supplementary/optional).

Makes sense, thanks for the edits Rob

@JamesHazelcast JamesHazelcast merged commit 084892b into hazelcast:main Feb 9, 2026
6 checks passed
@JamesHazelcast JamesHazelcast deleted the fix/ctt-820 branch February 9, 2026 10:56
This was referenced Feb 9, 2026
Merged
Merged
Merged
@github-actions
Copy link
Contributor

github-actions bot commented Feb 9, 2026

👍 Created #2098 to backport into v/5.5.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 9, 2026

👍 Created #2099 to backport into v/5.4.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 9, 2026

👍 Created #2100 to backport into v/5.6.

Rob-Hazelcast pushed a commit that referenced this pull request Feb 9, 2026
@github-actions @JamesHazelcast
Correct k8s rebalance delay advice [CTT-820] [v/5.4] (#2099)
29a0937 
Backport of #2090

`configuring-persistence` conflicted with `kubernetes-persistence` when
discussing `rebalance-delay-seconds` usage. The correct advice is from
`kubernetes-persistence`.

Fixes https://hazelcast.atlassian.net/browse/CTT-820

Co-authored-by: James Holgate <130981049+JamesHazelcast@users.noreply.github.com>
Rob-Hazelcast pushed a commit that referenced this pull request Feb 9, 2026
@github-actions @JamesHazelcast
Correct k8s rebalance delay advice [CTT-820] [v/5.6] (#2100)
173b02c 
Backport of #2090

`configuring-persistence` conflicted with `kubernetes-persistence` when
discussing `rebalance-delay-seconds` usage. The correct advice is from
`kubernetes-persistence`.

Fixes https://hazelcast.atlassian.net/browse/CTT-820

Co-authored-by: James Holgate <130981049+JamesHazelcast@users.noreply.github.com>
Rob-Hazelcast pushed a commit that referenced this pull request Feb 9, 2026
@github-actions @JamesHazelcast
Correct k8s rebalance delay advice [CTT-820] [v/5.5] (#2098)
0eb2181 
Backport of #2090

`configuring-persistence` conflicted with `kubernetes-persistence` when
discussing `rebalance-delay-seconds` usage. The correct advice is from
`kubernetes-persistence`.

Fixes https://hazelcast.atlassian.net/browse/CTT-820

Co-authored-by: James Holgate <130981049+JamesHazelcast@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gareth-johnston gareth-johnston gareth-johnston approved these changes

@Rob-Hazelcast Rob-Hazelcast Rob-Hazelcast approved these changes

Assignees

@JamesHazelcast JamesHazelcast

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@JamesHazelcast @gareth-johnston @Rob-Hazelcast